Skip to content

Conversation

Paulie-Aditya
Copy link
Contributor

Changes

Simply flipped over the conditions and that fixed it

in the left column: episodes 1 and 3
in the right column: episodes 2 and 4

Context

@To1ne brought this issue up once, that the video ordering on : https://git-scm.com/videos is wrong.
Specifically, episodes 2 and 4 are in the left column and the episodes 1 and 3 are in the right column.

… 1 and 3 on the left column, and 2 and 4 on the right column
@Paulie-Aditya
Copy link
Contributor Author

@To1ne Kindly let me know if the commit message is now appropriately defined, if not kindly guide me in the right direction for the same as well, would be more than happy to understand and contribute more meaningfully! Cheers!

@To1ne
Copy link
Collaborator

To1ne commented Sep 26, 2025

@Paulie-Aditya Let me suggest something like:

fix ordering of videos

The videos are shown in a grid view. This is done by showing them in two
columns. But during implementation the columns where swapped.

Reorder the columns so the top left video is the first of the series.

Some notes from this:

  • Start with a title, prefer to stay below 72 chars
  • Followed by an empty line
  • Then explain what the problem is (wrap lines at 80 chars)
  • Then explain what you're changing

But I've noticed something else. On mobile (small screens) the order is still wrong. We should switch to CSS grid or flexbox to make sure the order is persisted across viewports.

@Paulie-Aditya
Copy link
Contributor Author

Okay, will do the changes as instructed, thanks for the guidance!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants